Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Arista EOS show vrf to create a simple table of VRF names and RDs #449

Merged
merged 1 commit into from
Oct 30, 2019

Conversation

varnumd
Copy link

@varnumd varnumd commented Aug 6, 2019

ISSUE TYPE
  • New Template Pull Request
COMPONENT
  • arista_eos_show_vrf.template
  • arista_eos
  • show vrf
SUMMARY

A template for Arista EOS command "show vrf" which creates a table of the VRF names and Route Distinguishers (RD)


Copy link
Contributor

@FragmentedPacket FragmentedPacket left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@varnumd Thanks for submitting this PR!

Would you be able to just complete the data collection? Not too many other fields to account for.

@FragmentedPacket FragmentedPacket added the changes_requested Waiting on user to address feedback label Sep 27, 2019
@varnumd
Copy link
Author

varnumd commented Oct 1, 2019

@FragmentedPacket I'd be happy to. Would you mind sharing with me the requirements for data collection? This is the first time I'm submitting a change request with ntc and I'm not sure what I'm missing. Sorry for any troubles..

@FragmentedPacket
Copy link
Contributor

It would just be capturing the rest of the line:
blue 10.125.253.15:1 ipv4,ipv6 v4:routing, Vlan1006, Vlan2230, Vlan2231,

I don't think it would be too difficult to add Protocols, State, Interfaces as Values and capture them. Does that make sense?

@jmcgill298 jmcgill298 changed the base branch from master to arista-show-vrf October 30, 2019 04:01
@jmcgill298 jmcgill298 merged commit 22b094a into networktocode:arista-show-vrf Oct 30, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
changes_requested Waiting on user to address feedback
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants